-
Notifications
You must be signed in to change notification settings - Fork 535
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updated for shared memory support. #328
base: master
Are you sure you want to change the base?
Conversation
updated for shared memory support.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! One general comment that I have is that this changes the command line params for test_gdiplus.exe, so if we reference it anywhere in the documentation (IIRC we do in a couple of places, e.g. https://github.com/googleprojectzero/winafl/blob/master/readme_dr.md but possibly others), the documentation will be out-of-date and should be updated as well.
gdiplus.cpp
Outdated
IStream* stream = SHCreateMemStream(reinterpret_cast<BYTE*>(sample_bytes), sample_size); | ||
image = Gdiplus::Image::FromStream(stream); | ||
} | ||
if (image && (Ok == image->GetLastStatus())) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indentation seems to be off here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trying multiple options.. but for some reasons its just not working..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trying to avoid indentation issue..
gdiplus.cpp
Outdated
IStream* stream = SHCreateMemStream(reinterpret_cast<BYTE*>(sample_bytes), sample_size); | ||
image = Gdiplus::Image::FromStream(stream); | ||
} | ||
if (image && (Ok == image->GetLastStatus())) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trying multiple options.. but for some reasons its just not working..
For me, it shows as having a 2 tab indentation whereas the line before has a single tab. |
updated for shared memory support.